combobox: Stop using screen width/height
authorMatthias Clasen <mclasen@redhat.com>
Mon, 11 Apr 2016 03:42:57 +0000 (23:42 -0400)
committerMatthias Clasen <mclasen@redhat.com>
Thu, 28 Apr 2016 03:18:16 +0000 (23:18 -0400)
Popups should always be placed relative to a monitor.

gtk/gtkcombobox.c

index fe21a1fa9d580889bf2dd21302ba0054f2129d37..481c1808c64a2ad5ff396f4bec34b32c4a85af3b 100644 (file)
@@ -2007,10 +2007,12 @@ gtk_combo_box_menu_position_over (GtkMenu  *menu,
   GtkAllocation       content_allocation;
   GtkAllocation       child_allocation;
   GList              *children;
-  gint                screen_width;
   gint                menu_xpos;
   gint                menu_ypos;
   gint                menu_width;
+  GdkDisplay *display;
+  GdkMonitor *monitor;
+  GdkRectangle workarea;
 
   active = gtk_menu_get_active (GTK_MENU (priv->popup_widget));
 
@@ -2056,12 +2058,14 @@ gtk_combo_box_menu_position_over (GtkMenu  *menu,
                               &menu_xpos, &menu_ypos);
 
   /* Clamp the position on screen */
-  screen_width = gdk_screen_get_width (gtk_widget_get_screen (widget));
-  
-  if (menu_xpos < 0)
-    menu_xpos = 0;
-  else if ((menu_xpos + menu_width) > screen_width)
-    menu_xpos -= ((menu_xpos + menu_width) - screen_width);
+  display = gtk_widget_get_display (widget);
+  monitor = gdk_display_get_monitor_at_window (display, gtk_widget_get_window (widget));
+  gdk_monitor_get_workarea (monitor, &workarea);
+
+  if (menu_xpos < workarea.x)
+    menu_xpos = workarea.x;
+  else if ((menu_xpos + menu_width) > workarea.x + workarea.width)
+    menu_xpos -= (menu_xpos + menu_width) - (workarea.x + workarea.width);
 
   *x = menu_xpos;
   *y = menu_ypos;